Skip to content

fix(appellate): Tweaks ACMSPageHandler to handle firefox load - #422

Merged
mlissner merged 1 commit into
mainfrom
refactor-updates-logic-to-create-acms-observer
Feb 2, 2026
Merged

fix(appellate): Tweaks ACMSPageHandler to handle firefox load#422
mlissner merged 1 commit into
mainfrom
refactor-updates-logic-to-create-acms-observer

Conversation

@ERosendo

@ERosendo ERosendo commented Jan 30, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes ACMS page observer to handle Firefox page load timing and make sure observer is active for HTMX updates.

The existing ACMSPageHandler logic had a conditional that would skip observer creation if indexContent or fullDocketContent elements were already present on the page. This behavior worked in Chrome, where the extension typically loads before HTMX incrementally adds content, but broke in Firefox, where the page often finishes loading before the extension runs.

In Firefox this resulted in:

  • When content was already present, handleAcmsDocket() would run but no observer was created
  • Without an observer, subsequent HTMX partial updates were not detected
  • RECAP failing to respond to dynamic content changes after initial load

key changes:

  • Restructured the logic to always create the observer (if one does not already exist)
  • Preserved the immediate handleAcmsDocket() call for already-loaded content
  • Adds singleton pattern to prevent duplicate observers if ACMSPageHandler is called multiple times

@ERosendo
ERosendo force-pushed the refactor-updates-logic-to-create-acms-observer branch from 1f91f00 to 758384c Compare February 1, 2026 04:36
@ERosendo
ERosendo marked this pull request as ready for review February 1, 2026 05:07
@ERosendo
ERosendo requested a review from albertisfu February 1, 2026 05:07
@ERosendo ERosendo moved this to To Do in Sprint (Web Team) Feb 2, 2026
@mlissner
mlissner merged commit 5cee91e into main Feb 2, 2026
10 of 11 checks passed
@mlissner
mlissner deleted the refactor-updates-logic-to-create-acms-observer branch February 2, 2026 19:28
@github-project-automation github-project-automation Bot moved this from To Do to Done in Sprint (Web Team) Feb 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants